Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop #900

Merged
merged 20 commits into from
Dec 3, 2020
Merged

Develop #900

merged 20 commits into from
Dec 3, 2020

Conversation

PeterRao
Copy link
Collaborator

@PeterRao PeterRao commented Dec 2, 2020

feat: add listObjectV2 api
feat: add retryMax for when request error is net error or timeout
fix(browser): use shim process to solve setTimeout has min delay when browser tab is inactive
fix: after cancel multipartUpload, upload stream should be closed.
fix(browser): remove any-promise.
fix(browser): use shims/url fix node-url use split(/./)

beajer and others added 19 commits August 3, 2020 15:49
* feat: support non-english http header

* test: Add test cases for non-English request header information

* feat: add headerEncoding option

* feat: add encoder types

* fix: remove object spread syntax

Co-authored-by: biejia <wb-bj559483@alibaba-inc.com>
* feat: auto set stsConfig when sts expires

* test: add auto set sts test case
#887)

* fix: replace process for browser

* fix: use browserify options to replace process.js (#886)

Co-authored-by: beajer <919060679@qq.com>
* fix: merge conflict

* test: ResponseTimeoutError
* feat: retry when net err or timeout

* test: should not retry when params.stream is not readable

* test: retry in browser

Co-authored-by: beajer <919060679@qq.com>
* fix: getBucketVersions should use NextKeyMarker property

* docs: getBucketVersions should use NextKeyMarker property

* docs: Compatible wrong property
* fix: multipartUpload

zancun

* fix: test case

* test: multipartUploadStreams

* test: cov

* fix: error method

* fix: test and destory(err) when cancel

Co-authored-by: weiyie <912881342@qq.com>
# Conflicts:
#	lib/browser/client.js
#	lib/client.js
* feat: listObjectsV2

* style: trailingComma none
* fix: check region and endpoint for ssrf

* fix: retry in browser

Co-authored-by: beajer <919060679@qq.com>
@codecov
Copy link

codecov bot commented Dec 2, 2020

Codecov Report

Merging #900 (a6d11fd) into master (1b59e6f) will increase coverage by 0.31%.
The diff coverage is 98.03%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #900      +/-   ##
==========================================
+ Coverage   93.39%   93.70%   +0.31%     
==========================================
  Files          86       88       +2     
  Lines        2560     2655      +95     
==========================================
+ Hits         2391     2488      +97     
+ Misses        169      167       -2     
Impacted Files Coverage Δ
lib/common/object/getBucketVersions.js 92.10% <ø> (ø)
lib/common/utils/checkConfigValid.js 93.75% <93.75%> (ø)
lib/client.js 91.07% <94.73%> (+0.34%) ⬆️
lib/common/client/getReqUrl.js 91.89% <100.00%> (+0.46%) ⬆️
lib/common/client/initOptions.js 89.65% <100.00%> (+0.76%) ⬆️
lib/common/parallel.js 48.91% <100.00%> (+6.56%) ⬆️
lib/common/utils/retry.js 100.00% <100.00%> (ø)
lib/common/utils/setRegion.js 100.00% <100.00%> (ø)
lib/managed-upload.js 92.89% <100.00%> (+1.77%) ⬆️
lib/object.js 96.29% <100.00%> (+0.46%) ⬆️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b59e6f...67f0e5f. Read the comment docs.

@PeterRao PeterRao merged commit 5ed82a0 into master Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants